Fix parser naming conventions in documentation #127537
Conversation
|
@tur11ng : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 379ae31: ✅ Validation status: passed
For more details, please refer to the build report. |
|
@oshezaf - Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
prmerger-automator
bot
added
the
aq-pr-triaged
There was a problem hiding this comment.
Pull Request Overview
Updates documentation to standardize unifying parser naming conventions by adding the consistent Im prefix and correcting several parser identifiers.
- Standardizes parser names: imAuthentication -> _Im_Authentication, imFileEvent -> _Im_FileEvent, imRegistry -> _Im_Registry.
- Updates multi-value Process Event cell to use new names but removes prior list markers, potentially reducing clarity.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| | Network Session | _Im_NetworkSession | | ||
| | Process Event | - imProcessCreate<br> - imProcessTerminate | | ||
| | Registry Event | imRegistry | | ||
| | Process Event | _Im_ProcessCreate<br> _Im_ProcessTerminate | |
There was a problem hiding this comment.
[nitpick] The hyphen list markers were removed for the multi-parser cell, which may reduce readability and consistency compared to the previous format. Consider restoring list markers for each entry, e.g.: - _Im_ProcessCreate
- _Im_ProcessTerminate.
| | Process Event | _Im_ProcessCreate<br> _Im_ProcessTerminate | | |
| | Process Event | - _Im_ProcessCreate<br>- _Im_ProcessTerminate | |
Copilot uses AI. Check for mistakes.
|
Can you review this old PR and determine whether it needs to be closed or merged? @MicrosoftDocs/public-repo-pr-review-team |
No description provided.